Skip to content

ref(explore): Migrate context.tsx to nuqs#115435

Merged
ryan953 merged 1 commit into
masterfrom
ryan953/migrate-context-nuqs
May 13, 2026
Merged

ref(explore): Migrate context.tsx to nuqs#115435
ryan953 merged 1 commit into
masterfrom
ryan953/migrate-context-nuqs

Conversation

@ryan953
Copy link
Copy Markdown
Member

@ryan953 ryan953 commented May 12, 2026

Summary

  • Replaces updateNullableLocation with nuqs useQueryStates in useSetQueryParamsSavedQuery
  • Removes dependencies on useLocation, useNavigate, and updateNullableLocation

Test plan

  • Lint passes
  • Verify saving query params (id/title) works correctly in explore

…Location to nuqs

Replace the manual location/navigate approach with nuqs useQueryStates.
This removes the dependency on useLocation, useNavigate, and updateNullableLocation
in favor of nuqs parseAsString parsers for the id and title query params.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ryan953 ryan953 requested a review from a team as a code owner May 12, 2026 21:28
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.50%

@ryan953 ryan953 requested a review from a team May 12, 2026 21:49
@ryan953 ryan953 merged commit 092d115 into master May 13, 2026
75 of 76 checks passed
@ryan953 ryan953 deleted the ryan953/migrate-context-nuqs branch May 13, 2026 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants